Skip to content

ENH: Convert Filtering tests to GTest#5947

Open
hjmjohnson wants to merge 4 commits intoInsightSoftwareConsortium:mainfrom
hjmjohnson:convert-filtering-tests-to-gtest
Open

ENH: Convert Filtering tests to GTest#5947
hjmjohnson wants to merge 4 commits intoInsightSoftwareConsortium:mainfrom
hjmjohnson:convert-filtering-tests-to-gtest

Conversation

@hjmjohnson
Copy link
Member

Summary

  • Convert itkChangeLabelImageFilterTest (ImageLabel) to GTest
  • Convert itkCompositeValleyFunctionTest (BiasCorrection) to GTest
  • Convert itkGaborKernelFunctionTest (ImageSources) to GTest
  • Convert itkProjectedIterativeDeconvolutionImageFilterTest (Deconvolution) to GTest

Each conversion adds ITKGoogleTest to TEST_DEPENDS in the module's itk-module.cmake, removes the old CTest from the test driver, and adds a new GTest file with creategoogletestdriver().

Test plan

  • CI builds and all tests pass
  • New GTest tests are discovered and run via ctest
  • Existing tests in each module are unaffected

🤖 Generated with Claude Code

hjmjohnson and others added 4 commits March 13, 2026 19:20
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Enhancement Improvement of existing methods or implementation type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct area:Filtering Issues affecting the Filtering module labels Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Filtering Issues affecting the Filtering module type:Enhancement Improvement of existing methods or implementation type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Testing Ensure that the purpose of a class is met/the results on a wide set of test cases are correct

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant